Draft
Conversation
Signed-off-by: James M Snell <jasnell@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com> Assisted-by: Opencode:Opus 4.6
Signed-off-by: James M Snell <jasnell@gmail.com> Assisted-by: Opencode:Opus 4.6
Signed-off-by: James M Snell <jasnell@gmail.com> Assisted-by: Opencode:Opus 4.6
Signed-off-by: James M Snell <jasnell@gmail.com> Assisted-by: Opencode:Opus 4.6
Improve stream priority, headers support detection, and http3 session ticket app data - Replace sendOrder with proper priority/incremental support using packed uint32 encoding across the JS/C++ boundary - Rename StreamPriorityFlags to NON_INCREMENTAL/INCREMENTAL, fix the inverted incremental flag logic in the HTTP/3 application - Add Application::SupportsHeaders() virtual and HeadersSupportState session state flag to detect header support at enqueue time and hard-error at drain time if headers are unsupported - Implement HTTP/3 session ticket app data serialization with version byte, CRC32 integrity check, and big-endian encoding of six HTTP/3 settings for 0-RTT resumption validation - Return both urgency and incremental from GetStreamPriority via new StreamPriorityResult struct Signed-off-by: James M Snell <jasnell@gmail.com> Assisted-by: Opencode:Opus 4.6
Replace the unused sendOrder option with a working priority mechanism
that supports RFC 9218 urgency levels and the incremental flag.
Streams expose a setPriority({ level, incremental }) method and a
priority getter returning { level, incremental } or null. Priority
can also be set at stream creation time via options. Headers support
detection throws ERR_INVALID_STATE when attempting to send headers
on a non-header-supporting application protocol.
Signed-off-by: James M Snell <jasnell@gmail.com>
Assisted-by: Opencode:Opus 4.6
Signed-off-by: James M Snell <jasnell@gmail.com> Assisted-by: Opencode:Opus 4.6
Signed-off-by: James M Snell <jasnell@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com>
Signed-off-by: James M Snell <jasnell@gmail.com>
Collaborator
|
Review requested:
|
nodejs-github-bot
added
lib / src
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
WIP... finishing quic... more to do
Signed-off-by: James M Snell jasnell@gmail.com